Skip to content
This repository was archived by the owner on Mar 9, 2026. It is now read-only.

chore(ci): remove deprecated deny-licenses option from dependency-review#55

Merged
roottool merged 1 commit intomainfrom
chore/remove-deny-licenses-deprecated-option
Mar 1, 2026
Merged

chore(ci): remove deprecated deny-licenses option from dependency-review#55
roottool merged 1 commit intomainfrom
chore/remove-deny-licenses-deprecated-option

Conversation

@roottool
Copy link
Copy Markdown
Owner

@roottool roottool commented Mar 1, 2026

Description

Removes the deprecated deny-licenses option from the dependency-review workflow. The option is scheduled for removal in the next major release of actions/dependency-review-action.

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • Security fix (non-breaking change addressing a security issue)
  • Documentation (changes to documentation only)
  • Refactoring (non-breaking change improving code structure)
  • Tests (adding or updating tests)
  • Tooling / CI (changes to build tools, CI configuration)

Boundary Checklist (Required for Implementation Changes)

  • This is a non-implementation change (Documentation, Tooling, or CI only)
    • Reason: CI workflow change only — no parser logic, public API, or security rules affected.

Security & API Stability

Security Impact:

  • No impact on security (no changes to forbidden keys, prototype safety)
  • Reviewed against security rules in AGENTS.md
  • If security-related: Explain impact and mitigation

API Contract:

  • No changes to public API (parse function signature, type definitions)
  • No breaking changes to ParseResult, ParseIssue, or IssueCode
  • If API change: Is this a breaking change? (Yes/No + explanation)

Versioning:

  • Change is compatible with current v0.x versioning policy
  • If breaking change proposed: Justification for major version bump

Testing

Automated Checks:

  • TypeScript type checking passes (bun run check:type:source)
  • All tests pass (bun run test)
  • Build succeeds (bun run build)

Additional Verification:

CI-only change. No source code or tests affected.

Related


🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

The deny-licenses option is deprecated and may be removed in the next
major release of actions/dependency-review-action.

Ref: actions/dependency-review-action#938

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@roottool roottool self-assigned this Mar 1, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (df8d1e1) to head (3548006).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #55   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           22        22           
  Branches         6         6           
=========================================
  Hits            22        22           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@roottool roottool merged commit 5686c7a into main Mar 1, 2026
13 checks passed
@roottool roottool deleted the chore/remove-deny-licenses-deprecated-option branch March 1, 2026 06:34
@roottool roottool added the ci Some changes have been changed to CI. label Mar 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci Some changes have been changed to CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant